Skip to content

Docs: Added Typography and Icons to Foundation section - #254

Open
zoharma wants to merge 2 commits into
mainfrom
zma/ds-theme-docs-foundation-type-icons
Open

Docs: Added Typography and Icons to Foundation section #254
zoharma wants to merge 2 commits into
mainfrom
zma/ds-theme-docs-foundation-type-icons

Conversation

@zoharma

@zoharma zoharma commented Jul 7, 2026

Copy link
Copy Markdown
Contributor
  1. Typography
  2. Icons

This branch is dependent on #222 and related to future #209 (but not required)

@zoharma
zoharma force-pushed the zma/ds-theme-docs-foundation-type-icons branch from 8241328 to c617eb1 Compare July 30, 2026 10:36
@zoharma zoharma added the v0.6.3 label Jul 30, 2026
@zoharma
zoharma force-pushed the zma/ds-theme-docs-foundation-type-icons branch 3 times, most recently from d3a2843 to 54e3662 Compare July 30, 2026 13:24
@zoharma
zoharma marked this pull request as ready for review July 30, 2026 13:26
Comment thread src/storybook/foundation/2. typography.mdx Outdated
</td>
</tr>
<tr>
<td>Download</td>

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would these icons be used also for save/load? If so worth adding to the table as they are common in GDA at least

@zoharma zoharma Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wanted to show examples rather than showing a full list. I was also avoiding save on purpose as I find the disk anachronistic and not sure about it... (reasoning: modern software saves automatically, download shows differently, save to cloud also different, so it loses its meaning)

Comment thread src/storybook/foundation/3. icons.mdx Outdated
*/
declare module "@mui/material/styles" {
interface TypographyVariants {
h1Display: React.CSSProperties;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we have our own Typography storybook stories to demonstrate each variant along with a short note on how/when to use each? Or make the variants available in the existing MUI stories

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that is a good point - should I add one to this PR? or create a new one?

Comment thread src/storybook/foundation/2. typography.mdx Outdated
Comment thread src/themes/DiamondDSTheme.test.ts
Added:
2. typography
3. Icons
+
- Added icons pack to show icons
- Updated typography to align with #222
- Icons is standalone
@zoharma
zoharma force-pushed the zma/ds-theme-docs-foundation-type-icons branch from 54e3662 to e4c9212 Compare August 6, 2026 13:35
@zoharma
zoharma requested a review from a team August 6, 2026 13:35
@zoharma

zoharma commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

@douglaswinter Updated some of the wording around scanning and calm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants